Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create wireguard.root template #2249

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Create wireguard.root template #2249

merged 1 commit into from
Dec 13, 2024

Conversation

hotfur
Copy link
Contributor

@hotfur hotfur commented Nov 28, 2024

These two permission is absolutely required:

  • CAP_NET_ADMIN is needed for modifying routes.
  • CAP_NET_RAW is for modifying iptables.
    When the app starts to set up a tunnel, it seems to execute "cat /sys/module/wireguard/version" to check if wireguard kernel module is loaded or not. Despite the permission seems okay, without CAP_DAC_READ_SEARCH it could not read the version number and threw an error in the application log.
    CAP_DAC_OVERRIDE is needed optionally for installing Wireguard command line tools. It could be turned back off once the binaries have been copied.

@tiann tiann merged commit 8d31bd3 into tiann:main Dec 13, 2024
backslashxx pushed a commit to backslashxx/KernelSU that referenced this pull request Dec 13, 2024
These two permission is absolutely required:
- CAP_NET_ADMIN is needed for modifying routes.
- CAP_NET_RAW is for modifying iptables.
When the app starts to set up a tunnel, it seems to execute "cat
/sys/module/wireguard/version" to check if wireguard kernel module is
loaded or not. Despite the permission seems okay, without
CAP_DAC_READ_SEARCH it could not read the version number and threw an
error in the application log.
CAP_DAC_OVERRIDE is needed optionally for installing Wireguard command
line tools. It could be turned back off once the binaries have been
copied.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants